-
Notifications
You must be signed in to change notification settings - Fork 32
π fix wrong ServiceRunGet model
#7349
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
π fix wrong ServiceRunGet model
#7349
Conversation
Codecov ReportAll modified and coverable lines are covered by tests β
Additional details and impacted files@@ Coverage Diff @@
## master #7349 +/- ##
==========================================
+ Coverage 85.58% 88.55% +2.96%
==========================================
Files 1705 1364 -341
Lines 66224 56577 -9647
Branches 1124 481 -643
==========================================
- Hits 56681 50099 -6582
+ Misses 9223 6346 -2877
+ Partials 320 132 -188
Continue to review full report in Codecov by Sentry.
π New features to boost your workflow:
|
pcrespov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx
packages/models-library/src/models_library/api_schemas_webserver/resource_usage.py
Show resolved
Hide resolved
packages/models-library/src/models_library/api_schemas_webserver/resource_usage.py
Show resolved
Hide resolved
sanderegg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
|
dfb4149
into
ITISFoundation:master



What do these changes do?
In the previous PR β¨ Introduce wallet exchange ποΈΒ #7033 I refactored

But the model didn't have all the fields -> this is fixed in the PR
This was also partially hotfixed yesterday with this commit 6bc3d32
Related issue/s
How to test
Dev-ops checklist